Skip to content

chore: enable tokio macros and rt features#67

Open
bkfunk wants to merge 1 commit into
segfault87:mainfrom
bkfunk:chore/tokio-test-features
Open

chore: enable tokio macros and rt features#67
bkfunk wants to merge 1 commit into
segfault87:mainfrom
bkfunk:chore/tokio-test-features

Conversation

@bkfunk
Copy link
Copy Markdown

@bkfunk bkfunk commented May 13, 2026

Summary

The workspace-level tokio dependency currently enables only the io-util feature. This is sufficient for the library code paths, but consumers that want to write #[tokio::test]-style tests against these crates have to add their own tokio dependency with macros and rt features to make the attribute macros and runtime available.

Adding macros and rt to the workspace tokio features is a no-op for production code paths (those features are gated on use) but unblocks #[tokio::test] in downstream consumers and in any in-tree tests.

Change

-tokio = { version = "1.37", features = ["io-util"] }
+tokio = { version = "1.37", features = ["io-util", "macros", "rt"] }

Test plan

  • cargo check --workspace passes

Copilot AI review requested due to automatic review settings May 13, 2026 16:21
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Enables additional tokio features (macros and rt) at the workspace level so downstream consumers and in-tree tests can use #[tokio::test] without redeclaring tokio with extra features. No production code paths are affected since unused tokio features are gated.

Changes:

  • Add macros and rt to the workspace tokio feature list in Cargo.toml.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants